Skip to content

fix(page-router-outlet): actionBarVisibility not applied #1621

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 23, 2018

Conversation

ADjenkov
Copy link
Contributor

@ADjenkov ADjenkov commented Nov 20, 2018

actionBarVisibility property has no effect when applied to nested lazy loaded named outlet.

NSEmptyOutlet <page-router-outlet> should inherit its parent frame actionBarVisibility

e2e tests added for actionBarVisibility with lazy loaded named outlet

manoldonev
manoldonev previously approved these changes Nov 20, 2018
MartoYankov
MartoYankov previously approved these changes Nov 21, 2018
@ADjenkov ADjenkov dismissed stale reviews from MartoYankov and manoldonev via 571b39f November 21, 2018 15:05
@ghost ghost assigned SvetoslavTsenov Nov 22, 2018
@ADjenkov ADjenkov merged commit 6c91167 into master Nov 23, 2018
@ghost ghost removed bug in progress labels Nov 23, 2018
@ADjenkov ADjenkov deleted the djenkov/empty-outlet-action-bar branch November 23, 2018 09:04
SvetoslavTsenov pushed a commit that referenced this pull request Nov 26, 2018
* fix(p-r-o): actionBarVisibility has no effect when applied to nested lazy loaded named outlet

* tets: add tests for actionBarVisibility and lazy loaded named outlet

* chore: include images for ios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants